Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New "canard" button #758

Merged
merged 3 commits into from
Oct 31, 2024
Merged

New "canard" button #758

merged 3 commits into from
Oct 31, 2024

Conversation

xentenza
Copy link
Collaborator

An additional colour for the buttons, based on the graphic charter of the EPFL; red is sometimes perceived as "too aggressive".

An additional colour for the buttons, based on the graphic charter of the EPFL; red is sometimes perceived as "too aggressive".
Copy link

github-actions bot commented Oct 21, 2024

Test Results

  1 files    1 suites   0s ⏱️
281 tests 246 ✅ 0 💤  0 ❌ 35 🔥
281 runs  211 ✅ 0 💤 35 ❌ 35 🔥

For more details on these errors, see this check.

Results for commit 62ff987.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Oct 21, 2024

🔎 Download the Backstop report for this pull request (link valid for 90 days):

Copy link
Contributor

@alinekeller alinekeller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La couleur $leman n'a pas un contraste suffisant avec le texte blanc (voir le tableau dans le styleguide). Il faudrait soit utiliser la couleur $canard qui est ok, soit définir une variante $leman-dark qui aurait un contraste suffisant avec le blanc. Par exemple la teinte #00827C serait ok.

- switched $leman to $canard
- added $canard-dark color
@xentenza
Copy link
Collaborator Author

La couleur $leman n'a pas un contraste suffisant avec le texte blanc (voir le tableau dans le styleguide). Il faudrait soit utiliser la couleur $canard qui est ok, soit définir une variante $leman-dark qui aurait un contraste suffisant avec le blanc. Par exemple la teinte #00827C serait ok.

Caramba, comment ais-je pu oublier de vérifier ça? 🫣

Merci. Partons alors sur la couleur $canard. J'ai également défini une variante $canard-dark (spéciale Haloween) avec une teinte #004248. N'hésite pas à proposer mieux!

@xentenza xentenza requested a review from alinekeller October 30, 2024 09:12
@xentenza xentenza changed the title New "leman" button New "canard" button Oct 30, 2024
@alinekeller
Copy link
Contributor

Caramba, comment ais-je pu oublier de vérifier ça? 🫣

Merci. Partons alors sur la couleur $canard. J'ai également défini une variante $canard-dark (spéciale Haloween) avec une teinte #004248. N'hésite pas à proposer mieux!

C'est parfait ! Je valide la couleur canard de Halloween 😄
Je l'ai ajoutée dans le fichier colors.json afin qu'elle apparaisse dans la liste sur la page "Colors" du styleguide.

@williambelle williambelle merged commit 41eb556 into dev Oct 31, 2024
4 checks passed
@williambelle williambelle deleted the styleguide/leman-button branch October 31, 2024 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants